Skip to content

Add advanced alt-text demo fixture - #63

Closed
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-prepare-scanner-demo
Closed

Add advanced alt-text demo fixture#63
taarikashenafi wants to merge 1 commit into
mainfrom
taarikashenafi-prepare-scanner-demo

Conversation

@taarikashenafi

Copy link
Copy Markdown
Contributor

Summary

  • add a synthetic advanced fixture for keyword stuffing, contextually inaccurate alt text, and an accurate control
  • add a credential-free demo:verify command that emits machine-readable scanner-shaped findings with mocked model verdicts clearly labeled
  • add a credential-gated demo:live command that scans only the synthetic fixture with GitHub Models and optional Azure Vision enrichment
  • document exact credentials, modes, and demo commands

Technical boundaries

  • the five deterministic rules remain enabled and credential-free by default
  • keyword stuffing and generated remediation remain part of the opt-in alt-text-quality model rule
  • mocked output replaces only the external model response; extraction, rule mapping, and Finding emission use production code
  • no GitHub Models or Azure calls were made while preparing this PR

Validation

  • npm test (164 tests)
  • npm run typecheck
  • npm run build
  • npm run lint
  • npm run format:check
  • npm run demo:verify
  • verified npm run demo:live refuses to start without a model token

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 40132dd1-60d2-4441-ab8b-e8c5dc8e049e
Comment thread scripts/demo-support.ts
response.end(contents)
} catch (error) {
response.writeHead(500, {'Content-Type': 'text/plain; charset=utf-8'})
response.end(error instanceof Error ? error.message : String(error))
@taarikashenafi

Copy link
Copy Markdown
Contributor Author

Closing as superseded by #66. The reusable verifier and optional live-model concepts were adapted there onto the existing example/site-with-errors controlled fixture, together with all five deterministic rules, mocked contextual-quality and Azure evidence, and the Scanner v3.4.1 npm-loading workflow. The separate advanced page and image from this draft were intentionally not duplicated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants